Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ksni on linux #7

Closed
wants to merge 4 commits into from
Closed

Use ksni on linux #7

wants to merge 4 commits into from

Conversation

njust
Copy link
Contributor

@njust njust commented Jun 5, 2021

Hi, I updated the linux implementation to use ksni as discussed in #2. I tested it on linux Mint 20.1.

@olback
Copy link
Owner

olback commented Jun 19, 2021

Hi, I'd like to see the old implementation still available under a feature flag since ksni does not work in all environments.

@ancwrd1
Copy link

ancwrd1 commented Jul 27, 2021

Doesn't libappindicator use the same dbus SNI interface internally to show the tray icon?
@olback could you give an example where it won't work?

@olback
Copy link
Owner

olback commented Oct 9, 2021

@ancwrd1 It does not work when using Waybar.

Old impl:
20211010_000205

New impl:
20211009_235945

Well, it does work but the icon is missing, space is allocated, hint and click actions work just fine.

Sorry for the extremely late reply.

@njust
Copy link
Contributor Author

njust commented Oct 21, 2021

@olback can you provide the code for the both examples? Last time when I tried to load the tray icon with the old implementation via gresources I failed and opened this bug here #6

@olback
Copy link
Owner

olback commented Oct 22, 2021

@njust I ran the linux example. It uses a "named icon" instead of a gresource icon.

@olback
Copy link
Owner

olback commented Apr 29, 2023

Version 0.8.0 is now released and supports both ksni and libappindicator.

@olback olback closed this Apr 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants